Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: provider StateCommitment integration #11867

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

frisitano
Copy link
Contributor

@frisitano frisitano commented Oct 18, 2024

Dependencies

This PR builds on #11842 and as such it should be reviewed after #11842 has been merged.

Overview

This PR introduces the StateCommitment type in StateProviders such that the type can be used for state commitment operations. In this PR we do not utilise the types for state commitment operations, that will be implemented in follow up PR's.

ToLatestStateProviderRef trait

We introduce ToLatestStateProviderRef trait and implement it on DatabaseProvider such that we can instantiate a LatestStateProviderRef from a DatabaseProvider without leaking StateCommitment type data. We update components that instantiate LatestStateProviderRef objects from a DatabaseProvider to leverage this trait.

supports: #11830

@frisitano frisitano mentioned this pull request Oct 18, 2024
9 tasks
@frisitano frisitano changed the title Feat/provider state commitment feat: provider StateCommitment integration Oct 18, 2024
@frisitano frisitano changed the title feat: provider StateCommitment integration feat: provider StateCommitment integration Oct 18, 2024
@frisitano frisitano force-pushed the feat/provider-state-commitment branch from 871cbc7 to cb77bce Compare October 18, 2024 06:14
@frisitano frisitano force-pushed the feat/provider-state-commitment branch from cb77bce to 03ea70e Compare October 18, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant